Skip to content

ci: use deterministic generated branch for version alignment#514

Merged
louis-sanna-dev merged 1 commit intomainfrom
codex/fix-speakeasy-version-alignment
May 6, 2026
Merged

ci: use deterministic generated branch for version alignment#514
louis-sanna-dev merged 1 commit intomainfrom
codex/fix-speakeasy-version-alignment

Conversation

@louis-sanna-dev
Copy link
Copy Markdown
Contributor

@louis-sanna-dev louis-sanna-dev commented May 6, 2026

Fixes the SDK generation workflows so post-generation version alignment checks out the branch generated by the same workflow run, instead of searching for the latest PR by author.

Why:

  • Generated PR authors can vary, so gh pr list --author app/github-actions can miss the PR.
  • The pinned Speakeasy reusable workflow accepts a feature_branch input, so we set a deterministic per-run branch name with github.run_id and reuse the same value in the alignment job.
  • Existing token plumbing is unchanged: GH_TOKEN and checkout still use secrets.CLIENT_PIPELINE.

Updated workflows:

  • .github/workflows/sdk_generation_mistralai_sdk.yaml
  • .github/workflows/sdk_generation_mistralai_azure_sdk.yaml
  • .github/workflows/sdk_generation_mistralai_gcp_sdk.yaml

Validation:

  • git diff --check
  • YAML parse for all three changed workflows
  • Confirmed the pinned Speakeasy reusable workflow accepts feature_branch
  • Manually dispatched the workflow from this branch and confirmed the deterministic generated branch flow works end to end

@louis-sanna-dev louis-sanna-dev force-pushed the codex/fix-speakeasy-version-alignment branch from 3af78dc to e54e5ec Compare May 6, 2026 15:16
@louis-sanna-dev louis-sanna-dev force-pushed the codex/fix-speakeasy-version-alignment branch from e54e5ec to 1a5bca9 Compare May 6, 2026 15:19
@louis-sanna-dev
Copy link
Copy Markdown
Contributor Author

It works: see #515

@louis-sanna-dev louis-sanna-dev changed the title ci: use generated branch output for version alignment ci: use deterministic generated branch for version alignment May 6, 2026
Copy link
Copy Markdown
Contributor

@mistralai-nfau mistralai-nfau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤷

@louis-sanna-dev louis-sanna-dev merged commit b8e3a84 into main May 6, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants